Skip to content

feat(operator): add keepOnDelete support for artifact resources - #104

Merged
lbroudoux merged 1 commit into
reshaprio:mainfrom
Vaishnav88sk:feat-artifact-keep-on-delete
Nov 2, 2026
Merged

lbroudoux merged 1 commit into
reshaprio:mainfrom
Vaishnav88sk:feat-artifact-keep-on-delete

Conversation

@Vaishnav88sk

Copy link
Copy Markdown
Member

Description

Following up on #101 and @lbroudoux's suggestion, this PR adds the keepOnDelete configuration flag to artifact Custom Resources (Prompts, CustomTools, and Resources).

This aligns artifact lifecycle management with other reShapr resources (Service, Exposition, GatewayGroup, SecretSource).

Summary of Changes

  • API: Added keepOnDelete boolean property (default false) to PromptsSpec, CustomToolsSpec, and ResourcesSpec.
  • Operator: Updated BaseArtifactReconciler#cleanup to check isKeepOnDelete(resource) and skip remote artifact deletion when set to true.
  • CRDs: Added keepOnDelete to OpenAPI schemas in prompts.reshapr.io-v1.yml, customtools.reshapr.io-v1.yml, and resources.reshapr.io-v1.yml.
  • Documentation: Updated prompts-cr.md, customtools-cr.md, and resources-cr.md with the new field.
  • Tests: Added testCleanup_honorsKeepOnDelete in PromptsReconcilerTest.

Signed-off-by: Vaishnav88sk <vaishnavsk8804@gmail.com>
@Vaishnav88sk Vaishnav88sk added this to the 0.0.5 milestone Nov 2, 2026
@lbroudoux lbroudoux added enhancement New feature or request theme/operator labels Nov 2, 2026

@lbroudoux lbroudoux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@lbroudoux
lbroudoux merged commit 59ac72a into reshaprio:main Nov 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request theme/operator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants